Skip to content

fix(kubevirt): Fix VM migration with container disks (CVI, VI)#1007

Merged
yaroslavborbat merged 4 commits into
mainfrom
fix/kubevirt/container-disk-migrate
May 5, 2025
Merged

fix(kubevirt): Fix VM migration with container disks (CVI, VI)#1007
yaroslavborbat merged 4 commits into
mainfrom
fix/kubevirt/container-disk-migrate

Conversation

@yaroslavborbat
Copy link
Copy Markdown
Member

@yaroslavborbat yaroslavborbat commented May 5, 2025

Description

After mounting CVI and VI in read-only mode, we encountered a migration issue:
"Operation not supported: Cannot migrate empty or read-only disk ..."
This fix ensures proper handling of disk mounting modes during migration to prevent the error.
Screenshot From 2025-05-05 17-49-12

Why do we need it, and what problem does it solve?

What is the expected result?

Checklist

  • The code is covered by unit tests.
  • e2e tests passed.
  • Documentation updated according to the changes.
  • Changes were tested in the Kubernetes cluster manually.

Changelog entries

section: kubevirt 
type: fix
summary: Fix VM migration with container disks (CVI, VI)
impact_level: low

Signed-off-by: Yaroslav Borbat <yaroslav.borbat@flant.com>
Signed-off-by: Yaroslav Borbat <yaroslav.borbat@flant.com>
@danilrwx danilrwx self-requested a review May 5, 2025 12:41
danilrwx
danilrwx previously approved these changes May 5, 2025
Signed-off-by: Yaroslav Borbat <yaroslav.borbat@flant.com>
Signed-off-by: Yaroslav Borbat <yaroslav.borbat@flant.com>
@danilrwx danilrwx self-requested a review May 5, 2025 14:58
@yaroslavborbat yaroslavborbat merged commit 148e10c into main May 5, 2025
22 of 23 checks passed
@yaroslavborbat yaroslavborbat deleted the fix/kubevirt/container-disk-migrate branch May 5, 2025 15:08
yachmenevas pushed a commit that referenced this pull request Oct 15, 2025
Fix VM migration with container disks (CVI, VI)
---------
Signed-off-by: Yaroslav Borbat <yaroslav.borbat@flant.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants